Skip to content

Conversation

tyeth
Copy link
Member

@tyeth tyeth commented Sep 25, 2025

Adds the Full Fat Sparkle Motion

@tyeth tyeth marked this pull request as ready for review September 26, 2025 13:12
@tyeth tyeth requested a review from brentru September 26, 2025 15:54
Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tyeth just one question looks good and a standard new board PR aside from this

extern Wippersnapper WS; ///< Global member variable for callbacks

#endif // ADAFRUIT_WIPPERSNAPPER_H
/*!
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there a diff on this file?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bloody github web editor, my bad (for using it). It changes all the line endings sometimes, along with adding a newline at the end if it miss-understands the EOF/line-endings.
They suggested we add a .gitattributes file with config options for the repo, but I don't like the idea of forcing all text files to one of CRLF or LF, they have raised an issue internally as it used to work.
I will make a mental note and add a post it note about not using the web editor to my screen.

@tyeth tyeth force-pushed the add-sparkle-motion branch from 6be4cfc to 96a2277 Compare September 26, 2025 19:43
@tyeth tyeth force-pushed the add-sparkle-motion branch from 96a2277 to 7d1474d Compare September 26, 2025 19:57
Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tyeth tyeth merged commit eee3809 into main Sep 30, 2025
46 checks passed
@tyeth tyeth deleted the add-sparkle-motion branch September 30, 2025 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants